Skip to content

Fix #465#470

Merged
marci4 merged 1 commit into
TooTallNate:masterfrom
zhoulifu:issue465
May 3, 2017
Merged

Fix #465#470
marci4 merged 1 commit into
TooTallNate:masterfrom
zhoulifu:issue465

Conversation

@zhoulifu
Copy link
Copy Markdown
Contributor

@zhoulifu zhoulifu commented May 3, 2017

Prevent decoding handshake response after closing the WebSocketImpl

Prevent decoding handshake response after client closed
@zhoulifu
Copy link
Copy Markdown
Contributor Author

zhoulifu commented May 3, 2017

@Irahe PTAL

@Irahe
Copy link
Copy Markdown

Irahe commented May 3, 2017

Hello @zhoulifu , i'll merge this code to my lib and rebuild to test...
Thank you for your time.
i'm realy proud of this community!

👍

@Irahe
Copy link
Copy Markdown

Irahe commented May 3, 2017

Dear @zhoulifu , Unfortunately the fix wont work for me. I still have the same issues on firefox and Edge.

@zhoulifu
Copy link
Copy Markdown
Contributor Author

zhoulifu commented May 3, 2017

OMG, It's a mistaken identity, i mean to @marci4, HAHA
Sorry for disturbing you @Irahe XD

@marci4 marci4 merged commit 56e0d75 into TooTallNate:master May 3, 2017
@marci4 marci4 added this to the Release 1.3.4 milestone May 3, 2017
@zhoulifu zhoulifu deleted the issue465 branch May 4, 2017 15:50
@zhoulifu zhoulifu restored the issue465 branch May 4, 2017 15:50
@zhoulifu zhoulifu deleted the issue465 branch May 4, 2017 15:51
@marci4 marci4 added the Bug label May 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants